Skip to content

[IMP] Statistics: Add first format to bottombar#8560

Open
fdamhaut wants to merge 1 commit into
masterfrom
master-bottom-bar-less-simple-fix-flda
Open

[IMP] Statistics: Add first format to bottombar#8560
fdamhaut wants to merge 1 commit into
masterfrom
master-bottom-bar-less-simple-fix-flda

Conversation

@fdamhaut
Copy link
Copy Markdown
Contributor

@fdamhaut fdamhaut commented May 5, 2026

Task: 6186747

Description:

description of this task, what is implemented and why it is implemented that way.

Task: 6189449

review checklist

  • feature is organized in plugin, or UI components
  • support of duplicate sheet (deep copy)
  • in model/core: ranges are Range object, and can be adapted (adaptRanges)
  • in model/UI: ranges are strings (to show the user)
  • undo-able commands (uses this.history.update)
  • multiuser-able commands (has inverse commands and transformations where needed)
  • new/updated/removed commands are documented
  • exportable in excel
  • translations (_t("qmsdf %s", abc))
  • unit tested
  • clean commented code
  • track breaking changes
  • doc is rebuild (npm run doc)
  • status is correct in Odoo

@robodoo
Copy link
Copy Markdown
Collaborator

robodoo commented May 5, 2026

Pull request status dashboard

@fdamhaut fdamhaut force-pushed the master-bottom-bar-less-simple-fix-flda branch from 4921f37 to 9694976 Compare May 5, 2026 14:20
Before:
The bottom bar statistics seem to ignore the formats of  selected values entirely.
E.g. if you select dates, it will display their sum as a number (select 5/5/2026 and 6/5/2026, the statistics display 92294)

Now:
The statistics shows the format of the first cell (i.e. the anchor) of the selection this applies to SUM/AVG/MIN/MAX but not to COUNT/COUNT NUMBERS
If every cell in the selection is a date, hide the SUM operator

Task: 6186747
@fdamhaut fdamhaut force-pushed the master-bottom-bar-less-simple-fix-flda branch from 9694976 to f978e31 Compare May 18, 2026 12:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants